Skip to content

Small changes to GitHub templates#1516

Closed
SteveL-MSFT wants to merge 1 commit intoPowerShell:mainfrom
SteveL-MSFT:gitops-fixes
Closed

Small changes to GitHub templates#1516
SteveL-MSFT wants to merge 1 commit intoPowerShell:mainfrom
SteveL-MSFT:gitops-fixes

Conversation

@SteveL-MSFT
Copy link
Copy Markdown
Member

PR Summary

Changed based on copilot suggestions

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates GitHub Issue Forms templates under .github/ISSUE_TEMPLATE to slightly refine how guidance text and example outputs are displayed when users file issues.

Changes:

  • Adjusted the Feature Request placeholder YAML block style (>->) for the summary prompt.
  • Updated Bug Report textarea render values from console to shell for syntax highlighting consistency.
  • Removed trailing whitespace in blank lines within Bug Report placeholders.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/ISSUE_TEMPLATE/Feature_Request.yml Tweaks placeholder block scalar style for the feature request summary field.
.github/ISSUE_TEMPLATE/Bug_Report.yml Updates render language to shell and cleans up whitespace in example placeholders.

Copy link
Copy Markdown
Collaborator

@michaeltlombardi michaeltlombardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm surprised copilot recommended these changes. Except for the whitespace cleanup, the changes either don't affect the rendering (>- to >) or are semantically incorrect (console to shell).

new feature would solve. Try formulating it in user story style
(if applicable).
placeholder: >-
placeholder: >
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change removes the trimming of trailing spacing. It doesn't make a large difference here, but I'm not sure why copilot suggested it.

attributes:
label: Expected behavior
render: console
render: shell
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we actually want this change - console is accurate (we're asking users to submit the text from a terminal session, including both commands and output).

Shell highlighting will give POSIX shell syntax highlighting for the commands but may look strange for output and will incorrectly highlight when the user's shell was PowerShell.

@SteveL-MSFT
Copy link
Copy Markdown
Member Author

Let's just ignore these copilot suggestions

@SteveL-MSFT SteveL-MSFT closed this May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants